Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat Remove Microfrontend #831

Merged
merged 4 commits into from
Sep 14, 2023
Merged

Feat Remove Microfrontend #831

merged 4 commits into from
Sep 14, 2023

Conversation

varun2948
Copy link
Contributor

@varun2948 varun2948 commented Sep 14, 2023

I have removed fmtm_map micro frontend from docker and github actions please verify it @spwoodcock so that it wont break production and development and i have tested it in my local it worked fine.

Btw this PR have also added a feature onto it but you dont have to look all the changes you just have to look onto CONFIGS,DOCKER FILES, GITHUB ACTIONS FILE and ENV.

@varun2948 varun2948 temporarily deployed to 831/merge September 14, 2023 08:15 — with GitHub Actions Inactive
@varun2948 varun2948 self-assigned this Sep 14, 2023
@varun2948 varun2948 added enhancement New feature or request priority:critical Blocking current tasks or user workflow devops Related to deployment or configuration labels Sep 14, 2023
@spwoodcock
Copy link
Member

Awesome, so the microfrontend has been refactored to a single frontend now?

Copy link
Member

@spwoodcock spwoodcock left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great refactor - looks good to me - thanks for checking 👍

@varun2948
Copy link
Contributor Author

Awesome, so the microfrontend has been refactored to a single frontend now?

yeah just took some time to fix it cause it was making so much problem for any other contributors to work on frontend now it will be quite easy @spwoodcock

@robsavoye
Copy link
Collaborator

Great idea to make the code similar and easier to maintain !

@varun2948 varun2948 changed the title Feat projectinfo bug Feat Remove Microfrontend Sep 14, 2023
@spwoodcock spwoodcock merged commit 15a32d8 into development Sep 14, 2023
1 check failed
@spwoodcock spwoodcock deleted the feat-projectinfo-bug branch September 14, 2023 09:47
@varun2948
Copy link
Contributor Author

Just found a build issue on fmtm_main will look into it @spwoodcock

@spwoodcock
Copy link
Member

The deploy seemed to go ok after I just tweaked dockerfile.prod.

Is the error preventing a build for you?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
devops Related to deployment or configuration enhancement New feature or request priority:critical Blocking current tasks or user workflow
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants